-
Notifications
You must be signed in to change notification settings - Fork 102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Millan thickness and velocity to the shop #1443
Conversation
Hello @fmaussion! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2022-09-05 07:37:06 UTC |
Thanks @fmaussion this is awesome |
Yes in the end it was quite some work, partly because of some issues in the orginal dataset but also because this needs to be done with care... The kind of tasks I'd happily hand over to the community one day 😅 |
Thanks so much @fmaussion! |
BTW ... to use this I just want to point out that you need a new version of shapely > 1.7.1! update your environment might be best. |
Closes #1401
whats-new.rst